Skip to content

Fix version bugs#153

Merged
krisztianfekete merged 2 commits into
mainfrom
fix/version-flag
May 21, 2026
Merged

Fix version bugs#153
krisztianfekete merged 2 commits into
mainfrom
fix/version-flag

Conversation

@krisztianfekete
Copy link
Copy Markdown
Contributor

@krisztianfekete krisztianfekete commented May 21, 2026

What

agentevals --version has been printing 0.0.0-dev on every published release since the PyPI rename to agentevals-cli.

While in there:

  • Switched to hatch-vcs so the version comes straight from git tags.
  • Dropped the manual uv version "$VERSION" step in release.yml.
  • CI checkouts got fetch-depth: 0 so hatch-vcs can see tags.

Added

  • Two regression tests in tests/test_cli.py
  • Sidebar version span dropped to 0.5rem so long hatch-vcs dev versions like v0.9.2.dev0+g<sha> fit on a single line in the UI footer.

Fixes #152

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes the CLI/package version reporting after the PyPI distribution rename to agentevals-cli by aligning the runtime metadata lookup and switching packaging to VCS-derived versions.

Changes:

  • Update agentevals.__version__ to resolve from the installed agentevals-cli distribution and add regression tests for both module and CLI --version.
  • Move to hatch-vcs (dynamic = ["version"]) and remove the manual uv version ... step from the release workflow; ensure CI/release checkouts fetch full history/tags.
  • UI tweak to shrink the displayed version text in the sidebar footer.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/agentevals/__init__.py Fix version lookup to use the published distribution name (agentevals-cli).
tests/test_cli.py Add regression tests to ensure module/CLI versions don’t fall back to 0.0.0-dev.
pyproject.toml Switch to hatch-vcs with a dynamic version sourced from git tags.
.github/workflows/release.yml Remove manual version injection; fetch full git history for tag-based versioning.
.github/workflows/ci.yml Fetch full git history in CI so VCS-based version resolution can see tags.
.github/workflows/publish-evaluator-sdk.yml Fetch full git history (tags) for the SDK publishing workflow.
ui/src/components/sidebar/Sidebar.tsx Reduce footer version label font size to fit long dev version strings.
uv.lock Lockfile updates reflecting the dynamic version change and extras ordering.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/publish-evaluator-sdk.yml Outdated
Comment thread ui/src/components/sidebar/Sidebar.tsx
Comment thread pyproject.toml
@krisztianfekete krisztianfekete requested a review from peterj May 21, 2026 13:26
@krisztianfekete krisztianfekete merged commit 65834e7 into main May 21, 2026
5 checks passed
@krisztianfekete krisztianfekete deleted the fix/version-flag branch May 21, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

agentevals --version shows wrong version installed

3 participants